Skip to content

ci: Fix onnx_model_store resnet_v1 name issues#8255

Merged
mc-nv merged 4 commits into
r25.06from
yinggeh-DLIS-8365-fix-resnet-model-names
Jun 18, 2025
Merged

ci: Fix onnx_model_store resnet_v1 name issues#8255
mc-nv merged 4 commits into
r25.06from
yinggeh-DLIS-8365-fix-resnet-model-names

Conversation

@yinggeh

@yinggeh yinggeh commented Jun 17, 2025

Copy link
Copy Markdown
Contributor

What does the PR do?

Fix onnx_model_store resnet_v1_50 model name and input/output names.

Checklist

  • PR title reflects the change and is of format <commit_type>: <Title>
  • Changes are described in the pull request.
  • Related issues are referenced.
  • Populated github labels field
  • Added test plan and verified test passes.
  • Verified that the PR passes existing CI.
  • Verified copyright is correct on all changed files.
  • Added succinct git squash message before merging ref.
  • All template sections are filled out.
  • Optional: Additional screenshots for behavior/output changes with before/after.

Commit Type:

Check the conventional commit type
box here and add the label to the github PR.

  • ci

Related PRs:

Where should the reviewer start?

Test plan:

L0_long_running_stress--base

  • CI Pipeline ID:
    30314065

Caveats:

Background

Related Issues: (use one of the action keywords Closes / Fixes / Resolves / Relates to)

  • closes GitHub issue: #xxx

@yinggeh yinggeh self-assigned this Jun 17, 2025
@yinggeh yinggeh added the PR: ci Changes to our CI configuration files and scripts label Jun 17, 2025
@yinggeh
yinggeh marked this pull request as draft June 17, 2025 02:02
@yinggeh
yinggeh requested a review from mc-nv June 17, 2025 18:48
@yinggeh
yinggeh marked this pull request as ready for review June 17, 2025 18:50
mc-nv
mc-nv previously approved these changes Jun 17, 2025
@mc-nv
mc-nv merged commit 1d430f6 into r25.06 Jun 18, 2025
3 checks passed
@mc-nv
mc-nv deleted the yinggeh-DLIS-8365-fix-resnet-model-names branch June 18, 2025 20:35
mc-nv added a commit that referenced this pull request Jun 27, 2025
Co-authored-by: Misha Chornyi <99709299+mc-nv@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: ci Changes to our CI configuration files and scripts

Development

Successfully merging this pull request may close these issues.

2 participants